release: prepare unified signed runtime activation#6
Conversation
|
Warning You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again! |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 4b924a5c16
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 4b924a5c16
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
@codex review Please review exact final commit |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 12cbb801df
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
Codex Review: Didn't find any major issues. Swish! Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
Summary
Prepares the single public
agent-collabpackage for a future signed native-runtime activation without publishing private provider implementation. The change adds the closedstatus/prepare/login-groksetup client, exact activation-only third-party legal and SPDX evidence, stricter signed-archive verification, GitHub-hosted arm64 release validation, and the breaking public skill rename fromai-merge-resolvetomerge-resolve.This PR does not activate or publish the native runtime. Apple Developer Program approval, a Developer ID Application identity, signing, notarization, and final public artifact verification remain external release gates. Policy-only 3.1.0 behavior remains fail closed.
Companion private producer PR: sumitake/agent-collab-workspace#1882, final head
bd75fdd. Merge order: workspace producer first, then this public preparation PR. A later signed activation release is required before local legacy packages are removed.Boundary declaration
Generated and release surfaces
SKILL.mdfiles are in parity.changelog.d/fragments are present; generatedCHANGELOG.mdremains untouched under the fragment-only convention.Verification
python3 scripts/build_skills.py --checkpython3 scripts/build_marketplace.py --checkpython3 scripts/build-changelog.py --check— intentionally reports the uncompiled fragments;python3 scripts/build-changelog.py --dry-runpassed and the generated file is reserved for the release compiler.python3 -m unittest discover -s tests -t . -v— 247 passed on head57adf77e468b555cb514315b96205691d4ebd207.python3 -m unittest discover -s scripts -p 'test_*.py' -v— 254 passed.python3 scripts/check_release_consistency.py— 3.1.0 consistent.python3 scripts/secret_scan.py— 174 public source files clean.python3 scripts/check-public-export-safety.py --active-tree --history— safe.git diff --checkAdditional evidence:
runtime_setup.pyis a required canonical member with source byte/mode parity; removing it fails closed. Fresh-module regressions prove the public client loads without the POSIX-onlypwdmodule, omits caller-controlledHOME, and returns typedplatform_unsupportedbefore UID-dependent identity checks when POSIX UID APIs are unavailable.bd75fddreturned the exact closed management status response, bound the verified distribution to the unique loader-consumed Mach-O payload section, and passed strict dependency validation across 37 Mach-O members. It remains diagnostic only and is not a public release artifact.PROCEED. The later non-POSIXpwd/UID guards are covered by deterministic tests and CI; a current-head managed-Grok retry could not safely launch because this Codex sandbox cannot write the canonical managed Grok lock/auth lineage, and no raw or alternate-auth fallback was used.Review and post-condition
Tier 3: runtime verification, setup/authentication boundary, release supply chain, and workflow changes. The xAI-family review is independent of the OpenAI-family author and converged on
PROCEEDthrough the archive fix; the later portability patch remains deterministic-test and CI verified. Operator-reserved workflow/release paths are covered by the operator's session-specific pre-authorization; the operator remains the merge authority.Post-merge, policy-only installation remains the only supported public state. Final activation, tag/release publication, and local legacy-package removal stay blocked until the separately built runtime is Developer ID signed, notarized, packaged with exact reviewed metadata/legal members, and verified by the public release workflow.
Compliance trace
author: Codex
standing_directives: public policy-only boundary, no provider source or raw recipe, fragment-only changelog, exact signed-runtime verification, retirement state preserved
tier: 3
cross_check: managed Grok 4.5 xAI segmented review VERDICT PROCEED through the archive fix; exact 57adf77 non-POSIX identity guard is deterministic-test verified after the same-family P2; current xAI retry remains sandbox-blocked with no raw fallback
post_condition: 247 repository tests and 254 script tests pass on 57adf77; policy-only 3.1.0 remains fail closed; signed activation and legacy-package removal remain blocked on Apple Developer ID signing and notarization
mcp_coverage_gap: NONE
contributor_rights: OWNER-AUTHORED
operator_reserved: yes